Fix typos in numerous files #1875
Conversation
…and code to ensure consistency across all scripts and tests. Removed extra whitespace.
…ss all scripts and test.
Fixed files may also have extraneous whitespace removed.
SamErde
requested review from
SydneyhSmith,
adityapatwardhan,
alerickson and
anamnavi
as code owners
There was a problem hiding this comment.
Pull Request Overview
This pull request addresses typos and spelling errors across documentation, test files, and source code, improving consistency and clarity throughout the codebase.
- Fixed spelling errors in test descriptions and variable names, particularly changing "nonexistant" to "nonexistent" across multiple test files
- Corrected numerous spelling errors in documentation files, changelog entries, and source code comments
- Updated variable names from incorrect spellings (e.g., "delimeter" to "delimiter", "respository" to "repository")
Reviewed Changes
Copilot reviewed 62 out of 62 changed files in this pull request and generated 10 comments.
Show a summary per file
| File | Description |
|---|---|
| test/*.Tests.ps1 | Fixed spelling of "nonexistant" to "nonexistent" in test descriptions and variable names |
| src/code/*.cs | Corrected spelling errors in comments, variable names, and error messages |
| CHANGELOG/preview.md | Fixed typos in changelog entries including "Ingetration" to "Integration" |
| src/Microsoft.PowerShell.PSResourceGet.psd1 | Corrected typos in module manifest changelog |
| Examples/*.md | Updated example documentation to use correct spelling |
| buildtools.psd1 | Fixed "utilties" to "utilities" in module description |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
Looks good, there's just one failing test so requesting one change to fix that. The file 'VaryingCaseSensisityKeysScript.ps1' under 'test\testFiles\testScripts' needs to be updated to 'VaryingCaseSensitivityKeysScript.ps1'. You've corrected the spelling in the tests, but the file itself just needs to be updated.
Reflects fixed typo in other scripts that call this one.
Great catch! I just fixed that filename to match. |
|
Thanks for the PR @SamErde! |
3 participants
PR Summary
This pull request primarily addresses documentation and comment corrections across the codebase, focusing on fixing typos, clarifying instructions, and improving consistency in terminology.
Documentation and Instruction Improvements:
CHANGELOG/preview.mdandsrc/Microsoft.PowerShell.PSResourceGet.psd1files, such as fixing "Ingetration" to "Integration", "prelease" to "prerelease", and "Constrained Languange Mode" to "Constrained Language Mode". [1] [2] [3] [4] [5] [6]Example and Usage Documentation Fixes:
Examples/FindRepositorySearchingExamples.mdto consistently use "NonExistent" instead of "NonExistant" in tags, command names, and DSC resource names, improving clarity and accuracy in example scenarios. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12]Code Comment and Identifier Corrections:
CompressPSResource.cs.ContainerRegistryResponseUtil.cs.buildtools.psd1.PR Context
This PR mostly affects typos in documentation and inline comments. However, there were some typos in the actual source code that were addressed as well. In most cases, I believe the typos will have no material change, as some typos were consistently found across the codebae. However, there were some outlier typos that re-align the code with other properly spelled values.
This PR fixes #1874, but could potentially fix other unknown issues that were caused by typos.
PR Checklist
.h,.cpp,.cs,.ps1and.psm1files have the correct copyright headerWIP:or[ WIP ]to the beginning of the title (theWIPbot will keep its status check atPendingwhile the prefix is present) and remove the prefix when the PR is ready.